OCPBUGS-56202: DownStream Merge [07-25-2025]#2693
Merged
openshift-merge-bot[bot] merged 67 commits intomasterfrom Aug 6, 2025
Merged
OCPBUGS-56202: DownStream Merge [07-25-2025]#2693openshift-merge-bot[bot] merged 67 commits intomasterfrom
openshift-merge-bot[bot] merged 67 commits intomasterfrom
Conversation
There is no need to flush the chain before removing. Additionaly handle the case where the chain was already removed. Signed-off-by: Patryk Diak <pdiak@redhat.com>
Signed-off-by: Enrique Llorente <ellorent@redhat.com>
Signed-off-by: Enrique Llorente <ellorent@redhat.com>
Signed-off-by: Enrique Llorente <ellorent@redhat.com>
Signed-off-by: Enrique Llorente <ellorent@redhat.com>
Signed-off-by: Enrique Llorente <ellorent@redhat.com>
This change replace the customa llocator with the ovnk allocator and also reserver the cluster IPs so test can just ask for another IP without the problem of cluster ip collision. Signed-off-by: Enrique Llorente <ellorent@redhat.com>
Signed-off-by: Enrique Llorente <ellorent@redhat.com>
Signed-off-by: Enrique Llorente <ellorent@redhat.com>
Signed-off-by: Enrique Llorente <ellorent@redhat.com>
Signed-off-by: Enrique Llorente <ellorent@redhat.com>
Make required objects public Signed-off-by: Nadia Pinaeva <n.m.pinaeva@gmail.com>
Make all fields and methods public for now. Create node/util package for shared functions. Signed-off-by: Nadia Pinaeva <n.m.pinaeva@gmail.com>
Signed-off-by: Nadia Pinaeva <n.m.pinaeva@gmail.com>
Signed-off-by: Nadia Pinaeva <n.m.pinaeva@gmail.com>
Signed-off-by: Nadia Pinaeva <n.m.pinaeva@gmail.com>
Update gatewayReady function to only return bool as it always returns nil error. Signed-off-by: Nadia Pinaeva <n.m.pinaeva@gmail.com>
no changes to the function Signed-off-by: Nadia Pinaeva <n.m.pinaeva@gmail.com>
Make SetBridgeNetworkOfPortsInternal actually internal, rename to setOfPatchPort as it only updates patchPort. Also rename setBridgeNetworkOfPorts to SetNetworkOfPatchPort for the same reason. Signed-off-by: Nadia Pinaeva <n.m.pinaeva@gmail.com>
These functions use bridge lock, will convert them to BridgeConfiguration methods later. Move test functions related to flow generation. Signed-off-by: Nadia Pinaeva <n.m.pinaeva@gmail.com>
The locking logic is slightly changed, because now bridge is only locked during flow generation and not for the whole openflow_manager update duration. Also only one bridge is now locked at a time. Signed-off-by: Nadia Pinaeva <n.m.pinaeva@gmail.com>
Split internal member into read-only and read-writable, make sure to use mutex in the second case. Rename some methods to remove unneeded "bridge" part of the name. Move GetGatewayIface logic to the bridgeconfig creation. Signed-off-by: Nadia Pinaeva <n.m.pinaeva@gmail.com>
Signed-off-by: Nadia Pinaeva <n.m.pinaeva@gmail.com>
syncFlows only directly uses already protected GetBridgeName() method for bridgeConfig, and flow updates should be protected by the flowMutex. So hopefully I am not breaking anything... Signed-off-by: Nadia Pinaeva <n.m.pinaeva@gmail.com>
It used to require addressManager to updateGatewayIPs only to get bridgeConfig from it. We can just give nodePortWatcher its own reference to the bridgeConfig. Signed-off-by: Nadia Pinaeva <n.m.pinaeva@gmail.com>
Signed-off-by: Nadia Pinaeva <n.m.pinaeva@gmail.com>
store Advertised values to the copy and not to the original object. isIPv6 should be true in ipv6 case. Signed-off-by: Nadia Pinaeva <n.m.pinaeva@gmail.com>
Signed-off-by: Dan Winship <danwinship@redhat.com>
refactor pkg/node bridgeconfig into its own package, fix the locking
Adapt e2e kind provider for kubevirt localnet tests
Contributor
|
@jcaamano: /override requires failed status contexts, check run or a prowjob name to operate on.
Only the following failed contexts/checkruns were expected:
If you are trying to override a checkrun that has a space in it, you must put a double quote on the context. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
Contributor
|
/override ci/prow/qe-perfscale-payload-control-plane-6nodes |
Contributor
|
@jcaamano: Overrode contexts on behalf of jcaamano: ci/prow/qe-perfscale-payload-control-plane-6nodes DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
Contributor
|
/test openshift-e2e-gcp-ovn-techpreview-upgrade |
Contributor
@jcaamano looks like maybe try again: /test openshift-e2e-gcp-ovn-techpreview-upgrade |
Contributor
|
/test ci/prow/qe-perfscale-aws-ovn-small-udn-density-l3 |
Contributor
|
@jcaamano: The specified target(s) for The following commands are available to trigger optional jobs: Use DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
Contributor
|
/test qe-perfscale-aws-ovn-small-udn-density-l3 |
Contributor
|
/test qe-perfscale-aws-ovn-small-udn-density-churn-l3 |
1 similar comment
Contributor
|
/test qe-perfscale-aws-ovn-small-udn-density-churn-l3 |
Contributor
|
/lgtm |
Contributor
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jcaamano, openshift-pr-manager[bot] The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
openshift-ci
bot
added
the
approved
Contributor
|
/override ci/prow/lint |
Contributor
|
/override ci/prow/qe-perfscale-payload-control-plane-6nodes |
Contributor
|
@jcaamano: /override requires failed status contexts, check run or a prowjob name to operate on.
Only the following failed contexts/checkruns were expected:
If you are trying to override a checkrun that has a space in it, you must put a double quote on the context. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
Contributor
|
@jcaamano: Overrode contexts on behalf of jcaamano: ci/prow/qe-perfscale-payload-control-plane-6nodes DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
Contributor
|
/override ci/prow/lint |
Contributor
|
@jcaamano: Overrode contexts on behalf of jcaamano: ci/prow/lint DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
Contributor
|
@openshift-pr-manager[bot]: Jira Issue OCPBUGS-56202: All pull requests linked via external trackers have merged: Jira Issue OCPBUGS-56202 has been moved to the MODIFIED state. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
Contributor
|
@openshift-pr-manager[bot]: The following tests failed, say
Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
Contributor
|
[ART PR BUILD NOTIFIER] Distgit: ovn-kubernetes-base |
Contributor
|
[ART PR BUILD NOTIFIER] Distgit: ovn-kubernetes-microshift |
Contributor
|
[ART PR BUILD NOTIFIER] Distgit: ose-ovn-kubernetes |
Contributor
|
Fix included in accepted release 4.20.0-0.nightly-2025-09-08-182033 |
Contributor
|
Fix included in accepted release 4.20.0-0.nightly-2026-03-14-030631 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
16 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Automated merge of upstream/master → master.